Greptile Summary

This PR adds MCP tool annotations to all 9 browser automation tools, providing semantic metadata that helps LLMs make safer and more informed tool execution decisions. The implementation correctly categorizes tools as read-only (extract, observe, get_url, screenshot) or destructive (navigate, act, agent, session_create, session_close) based on their actual behavior.

Key Changes:

• Extended ToolSchema type with optional annotations field in src/tools/tool.ts
• Updated tool registration in src/index.ts to pass annotations to Smithery server
• Added title, readOnlyHint, and destructiveHint annotations to all 9 tools
• Annotations align with tool behavior: read-only tools only observe/extract, destructive tools modify state

Benefits:

• LLMs can prioritize read-only tools for information gathering before taking actions
• Better tool selection based on operation safety
• Cleaner tool display with human-readable titles
• No runtime behavior changes, purely additive metadata

The implementation is clean, well-structured, and follows MCP SDK patterns correctly.

Confidence Score: 5/5

• This PR is safe to merge with no identified issues
• The changes are purely additive metadata that doesn't affect runtime behavior. All annotations correctly reflect tool behavior (read-only vs destructive), type definitions are properly extended, and integration follows MCP SDK patterns. No logic changes, no breaking changes, and implementation is consistent across all tools.
• No files require special attention

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant LLM
    participant MCPServer as MCP Server
    participant Smithery as Smithery Tool Handler
    participant Tool as Tool Implementation
    participant Browser as Browserbase

    Note over LLM,Browser: Tool Registration Flow
    MCPServer->>Tool: Load tool definitions
    Tool->>Tool: Add annotations (title, readOnlyHint/destructiveHint)
    Tool-->>MCPServer: Return ToolSchema with annotations
    MCPServer->>Smithery: Register tool with annotations
    Note over Smithery: Stores metadata:<br/>- name<br/>- description<br/>- inputSchema<br/>- annotations

    Note over LLM,Browser: Tool Execution Flow (with annotation awareness)
    LLM->>LLM: Analyze available tools with annotations
    Note over LLM: Prefers readOnlyHint tools for<br/>information gathering
    LLM->>MCPServer: Call tool (e.g., browserbase_stagehand_extract)
    MCPServer->>Smithery: Route to tool handler
    Smithery->>Tool: Execute with params
    Tool->>Browser: Perform operation (read/write based on annotation)
    Browser-->>Tool: Return result
    Tool-->>Smithery: Return ToolResult
    Smithery-->>MCPServer: Return content
    MCPServer-->>LLM: Return result with awareness of operation safety

Greptile found no issues!

From now on, if a review finishes and we haven't found any issues, we will not post anything, but you can confirm that we reviewed your changes in the status check section.

_{This feature can be toggled off in your Code Review Settings by deselecting "Create a status check for each PR".}